Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing requires in chef_provider_package_rbenvrubygems.rb #1

Closed
wants to merge 1 commit into from
Closed

Missing requires in chef_provider_package_rbenvrubygems.rb #1

wants to merge 1 commit into from

Conversation

gregkare
Copy link

@gregkare gregkare commented Jan 6, 2012

Fixes the following problem:

NameError: uninitialized constant Chef::Rbenv

fnichol added a commit that referenced this pull request Jan 23, 2012
@fnichol
Copy link
Contributor

fnichol commented Jan 23, 2012

Look like this has come up a couple of times now. I had this issue in a couple of other cookbooks due to the way/order that Chef loads in libraries/ code. Are you able to double check if the above patch gets the job done? Thanks!

@fnichol
Copy link
Contributor

fnichol commented Feb 20, 2012

I'm assuming we've addressed this issue. Thank you!

@fnichol fnichol closed this Feb 20, 2012
@gregkare
Copy link
Author

Sorry about the delay, I completely forgot about your message! Your solution works as well, I finally got around to trying it.

@fnichol
Copy link
Contributor

fnichol commented Feb 20, 2012

No problem, I know what that's like :)

koodough referenced this pull request in koodough/chef-rbenv Jul 5, 2013
Gentoo package support for rbenv install_pkgs
GUI pushed a commit to NREL-cookbooks/rbenv that referenced this pull request Jan 13, 2014
Merge current finchol into master
CloCkWeRX added a commit that referenced this pull request Jul 26, 2015
@lock
Copy link

lock bot commented May 31, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants